Skip to content

Hackathon analytics #381

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Benjtalkshow merged 99 commits into from
Feb 11, 2026
Merged

Hackathon analytics #381

Benjtalkshow merged 99 commits into from
Feb 11, 2026

Conversation

@Benjtalkshow
Copy link
Collaborator

@Benjtalkshow Benjtalkshow commented Feb 11, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features

    • Added error notifications displaying when analytics data fails to load on hackathon pages.

  • Bug Fixes

    • Improved fallback logic for retrieving organization metadata and links from multiple sources.

  • Style

    • Enhanced visual spacing consistency with minimum height adjustments in hackathon creation and loading states.

@Benjtalkshow
fix: modify api.ts
f4dcdc0
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
b90d4ed 
… mod

merge into main
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
9ea81a5 
… mod

merge into main
@Benjtalkshow
fix: remove google auth buttom
5be269f
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
490dcb2 
… mod

merge into main
@Benjtalkshow
fix: fixes responsive fixes on organization
842fd48
@Benjtalkshow
fix: minor fixes
596a7f8
@Benjtalkshow
fix: minor fixes
96fee24
@Benjtalkshow
fix: modify create organization
9dfb149
@Benjtalkshow
fix: modify create organization
a194d90
@Benjtalkshow
fix: fix organization permission
b2ceee0
@Benjtalkshow
fix: merge into main
9ea97d1
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
adb4629 
… organization-fixes

merge into main
@Benjtalkshow
fix: merge into main
0c11420
@Benjtalkshow
feat: hackathon overview page
1f5ec24
@Benjtalkshow
feat: hackathon overview page
bca1ef6
@Benjtalkshow
feat: implement participant overview
9202b4f
@Benjtalkshow
feat: implement participant overview
096f265
@Benjtalkshow
feat: implement resources tab
b3478d0
@Benjtalkshow
feat: implement the submission tab
e83a0be
@Benjtalkshow
feat: implement comment tab
398be9b
@Benjtalkshow
fix: implement provider for hackathon
de546b1
@Benjtalkshow
fix: implement provider for hackathon
0fd2690
@Benjtalkshow
fix: minor fixes
0d7417f
@Benjtalkshow
fix: merge branch 'main' of https://github.com/Benjtalkshow/boundless …
4d3efee 
…into organization-fixes
@Benjtalkshow
fix: hackathon banner
83893e4
@Benjtalkshow
fix: hackathon banner
8013d62
@Benjtalkshow
fix: fix hackthon conflict
3fb1323
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
b7fc94f 
… hackathon-details

fix: merge into main
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
0594ac8 
… hackathon-details

Merge into main
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
54bd42e 
… fix-blog

Merge into main
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
b79d391 
… fix-blog

Merge into main
@Benjtalkshow
fix: fix error in fetching team posts
5c906a5
@Benjtalkshow
feat: implement create team, get my team
b000eb7
@Benjtalkshow
feat: implement create team, get my team
8e95934
@Benjtalkshow
feat: implement hackathon project submission flow
59586e4
@Benjtalkshow
feat: implement voting for submission
d812fec
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
d501680 
… fix-hackathon

Merge into main
@Benjtalkshow
fix: team formation updates
e62d570
@Benjtalkshow
fix: fix conflict
15a581b
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
4d89af9 
… fix-hackathon

merge into main
@Benjtalkshow
fix: implement team invitation
2589770
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
ed26db6 
… fix-hackathon

merge into main
@Benjtalkshow
feat: hackathon submissions bulk actions, ranking and ui refinements
1d20c57
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
aa259cd 
… fix-hackathon

Merge into main
@Benjtalkshow
fix: implement empty state for hackathons page
18747e4
@Benjtalkshow
feat: Implement submission visibility and explore submissions
76a9124
@Benjtalkshow
feat: Implement winners tab for hackathon winners display
e0fb888
@Benjtalkshow
feat: implement hackathon analytics
2775ba8
@Benjtalkshow
fix: fix coderabbit corrections
199c698
@Benjtalkshow
fix: fix coderabbit corrections
e7e78dd
@Benjtalkshow
fix: fix coderabbit corrections
736b900
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
e92b634 
… hackathon-analytics

merge into main
@Benjtalkshow
fix: fix conflict
e289ce6
@Benjtalkshow
Merge branch 'main' of https://github.com/Benjtalkshow/boundless into…
4e45528 
… hackathon-analytics

merge into main
@Benjtalkshow
fix: fix organization settings data persistence, hackathondrafts and …
29c281d 
…population
@vercel
Copy link

vercel bot commented Feb 11, 2026

@Benjtalkshow is attempting to deploy a commit to the Threadflow Team on Vercel.

A member of the Team first needs to authorize it.

@Benjtalkshow Benjtalkshow merged commit 5a95aee into boundlessfi:main Feb 11, 2026
2 of 5 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The PR introduces error handling for hackathon analytics, refactors organization data structure by moving tagline and about fields into nested metadata, updates API response handling to remove extra nesting layers, and adds fallback logic to read from both direct fields and metadata sources.

Changes

Cohort / File(s) Summary
Analytics Error Handling
app/(landing)/organizations/[id]/hackathons/[hackathonId]/page.tsx		 Extended useHackathonAnalytics hook return type to include error property; wired error state into component to display alert UI when analytics loading fails.
Organization Data Structure
components/organization/OrganizationSettings.tsx, components/organization/tabs/ProfileTab.tsx		 Moved tagline and about fields from top-level to nested metadata object in organization settings. Updated ProfileTab to read from both direct fields and metadata fallbacks for backward compatibility.
API Response Unwrapping
hooks/use-hackathons.ts, lib/api/hackathons.ts		 Removed extra nesting layer from API responses by accessing response.data directly instead of response.data.data in draft-related operations; simplified pagination assignments.
Layout Improvements
components/organization/hackathons/new/NewHackathonTab.tsx		 Added minimum height constraint (min-h-[60vh]) to loading and error state containers for consistent vertical spacing.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • Hackathon analytics #378: Modifies the same useHackathonAnalytics hook return shape and error handling integration in the hackathon page component.

Suggested reviewers

  • 0xdevcollins

Poem

🐰 The analytics now gracefully tumble,
When errors appear, alerts don't stumble,
Data restructures in metadata's nest,
API responses unwrapped at their best!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0xdevcollins 0xdevcollins Awaiting requested review from 0xdevcollins 0xdevcollins is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Benjtalkshow